Looks like this PR is not ready to merge, because of the following issues:

• This PR is missing the 'stat: QA assured' label
• This PR is missing the required milestone or project

Please fix the issues and try again

If you have any trouble, please check the PR guidelines

⚠️ No Changeset found

Latest commit: 774ab7b

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Walkthrough

Simplified ABAC state management by removing loading states from the useIsABACAvailable hook and standardizing boolean checks across ABAC admin components. Updated hook consumption pattern to destructure the data property from useHasLicenseModule with fallback defaults instead of direct boolean comparisons.

Changes

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

• Verify that removing the loading state from useIsABACAvailable doesn't break any dependent UI or logic that previously handled loading states
• Confirm that the destructuring pattern with { data: hasABAC = false } correctly handles undefined data scenarios and maintains expected behavior
• Review AdminABACPage changes to ensure the removal of ContextualbarSkeletonBody doesn't degrade UX during async operations
• Check that simplified boolean checks (!isABACAvailable vs !== true) are semantically equivalent in all contexts

Possibly related PRs

• feat: ABAC Rooms attributes tab #37408: Added the original ABAC room-attributes feature and useIsABACAvailable hook that these changes modify
• feat: add ABAC admin settings #37139: Introduced or modified ABAC admin components that are being refactored in this PR

Suggested labels

stat: ready to merge, stat: QA assured

Suggested reviewers

• aleksandernsilva
• tassoevan

Poem

🐰 Loading states trimmed clean and neat,
Boolean checks made pure and sweet,
Hooks restructured, logic refined,
ABAC controls now redesigned! ✨

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

📦 Docker Image Size Report

📈 Changes

Service	Current	Baseline	Change	Percent
sum of all images	1.2GiB	1.2GiB	+12MiB
rocketchat	358MiB	347MiB	+12MiB
omnichannel-transcript-service	132MiB	132MiB	+10KiB
queue-worker-service	132MiB	132MiB	+9.8KiB
ddp-streamer-service	127MiB	127MiB	+7.0KiB
account-service	114MiB	114MiB	+8.7KiB
authorization-service	111MiB	111MiB	+58KiB
stream-hub-service	111MiB	111MiB	+8.2KiB
presence-service	111MiB	111MiB	+8.4KiB

📊 Historical Trend

---
config:
  theme: "dark"
  xyChart:
    width: 900
    height: 400
---
xychart
  title "Image Size Evolution by Service (Last 30 Days + This PR)"
  x-axis ["11/15 22:28", "11/16 01:28", "11/17 23:50", "11/18 22:53", "11/19 23:02", "11/21 16:49", "11/24 16:27", "11/24 16:49 (PR)"]
  y-axis "Size (GB)" 0 --> 0.5
  line "account-service" [0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11]
  line "authorization-service" [0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11]
  line "ddp-streamer-service" [0.12, 0.12, 0.12, 0.12, 0.12, 0.12, 0.12, 0.12]
  line "omnichannel-transcript-service" [0.14, 0.14, 0.14, 0.14, 0.14, 0.13, 0.13, 0.13]
  line "presence-service" [0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11]
  line "queue-worker-service" [0.14, 0.14, 0.14, 0.14, 0.14, 0.13, 0.13, 0.13]
  line "rocketchat" [0.36, 0.36, 0.35, 0.35, 0.35, 0.34, 0.34, 0.35]
  line "stream-hub-service" [0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11]

Statistics (last 7 days):

• 📊 Average: 1.5GiB
• ⬇️ Minimum: 1.2GiB
• ⬆️ Maximum: 1.6GiB
• 🎯 Current PR: 1.2GiB

Updated: Mon, 24 Nov 2025 16:49:46 GMT